sizerequestcache: Make clear_cache clear all the cache
authorBenjamin Otte <otte@redhat.com>
Tue, 13 Nov 2012 20:51:29 +0000 (21:51 +0100)
committerBenjamin Otte <otte@redhat.com>
Wed, 14 Nov 2012 00:55:28 +0000 (01:55 +0100)
gtk/gtksizerequest.c
gtk/gtksizerequestcache.c
gtk/gtksizerequestcacheprivate.h

index 51043edb90263f72d05504fe5270e2c849caf218..eff854f20017b943dade7376b48da5a3a24a1c3f 100644 (file)
@@ -95,11 +95,9 @@ init_cache (GtkWidget        *widget)
 
   cache = _gtk_widget_peek_request_cache (widget);
 
-  if (_gtk_widget_get_width_request_needed (widget))
-    _gtk_size_request_cache_clear (cache, GTK_SIZE_GROUP_HORIZONTAL);
-  
-  if (_gtk_widget_get_height_request_needed (widget))
-    _gtk_size_request_cache_clear (cache, GTK_SIZE_GROUP_VERTICAL);
+  if (_gtk_widget_get_width_request_needed (widget) ||
+      _gtk_widget_get_height_request_needed (widget))
+    _gtk_size_request_cache_clear (cache);
 
   return cache;
 }
index d098e402f0a6268b769b26cf06a23c9c4bf8301b..f7597ebb78f9602d9bca8e5d4ebb6d7d0c975380 100644 (file)
@@ -31,46 +31,31 @@ _gtk_size_request_cache_init (SizeRequestCache *cache)
   memset (cache, 0, sizeof (SizeRequestCache));
 }
 
+static void
+free_sizes (SizeRequest **sizes)
+{
+  gint i;
+
+  for (i = 0; i < GTK_SIZE_REQUEST_CACHED_SIZES && sizes[i] != NULL; i++)
+    g_slice_free (SizeRequest, sizes[i]);
+      
+  g_slice_free1 (sizeof (SizeRequest *) * GTK_SIZE_REQUEST_CACHED_SIZES, sizes);
+}
+
 void
 _gtk_size_request_cache_free (SizeRequestCache *cache)
 {
-  _gtk_size_request_cache_clear (cache, GTK_SIZE_GROUP_HORIZONTAL);
-  _gtk_size_request_cache_clear (cache, GTK_SIZE_GROUP_VERTICAL);
+  if (cache->widths)
+    free_sizes (cache->widths);
+  if (cache->heights)
+    free_sizes (cache->heights);
 }
 
 void
-_gtk_size_request_cache_clear (SizeRequestCache *cache,
-                              GtkSizeGroupMode  orientation)
+_gtk_size_request_cache_clear (SizeRequestCache *cache)
                                
 {
-  SizeRequest **sizes;
-  gint          i;
-
-  if (orientation == GTK_SIZE_GROUP_HORIZONTAL)
-    {
-      sizes = cache->widths;
-
-      cache->widths            = NULL;
-      cache->cached_widths     = 0;
-      cache->last_cached_width = 0;
-      cache->cached_base_width = FALSE;
-    }
-  else
-    {
-      sizes = cache->heights;
-
-      cache->heights            = NULL;
-      cache->cached_heights     = 0;
-      cache->last_cached_height = 0; 
-      cache->cached_base_height = FALSE;
-   }
-
-  if (sizes)
-    {
-      for (i = 0; i < GTK_SIZE_REQUEST_CACHED_SIZES && sizes[i] != NULL; i++)
-       g_slice_free (SizeRequest, sizes[i]);
-      
-      g_slice_free1 (sizeof (SizeRequest *) * GTK_SIZE_REQUEST_CACHED_SIZES, sizes);
-    }
+  _gtk_size_request_cache_free (cache);
+  _gtk_size_request_cache_init (cache);
 }
 
index 939f9cb61fbd746575489643b0f999b85d838a32..60c061bdfef2191ba8a8514ad2aaaeb2f5abd0b3 100644 (file)
@@ -26,7 +26,6 @@
 #define __GTK_SIZE_REQUEST_CACHE_PRIVATE_H__
 
 #include <glib.h>
-#include <gtk/gtkenums.h>
 
 G_BEGIN_DECLS
 
@@ -68,8 +67,7 @@ typedef struct {
 void            _gtk_size_request_cache_init                    (SizeRequestCache       *cache);
 void            _gtk_size_request_cache_free                    (SizeRequestCache       *cache);
 
-void            _gtk_size_request_cache_clear                   (SizeRequestCache       *cache,
-                                                                GtkSizeGroupMode        orientation);
+void            _gtk_size_request_cache_clear                   (SizeRequestCache       *cache);
 
 G_END_DECLS